Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lib: accept Python 3 after Python 2 #1910

Closed
wants to merge 1 commit into from
Closed

lib: accept Python 3 after Python 2 #1910

wants to merge 1 commit into from

Conversation

sam-github
Copy link
Contributor

Python 3 is allowed as a compatible Python, but its looked for after
Python 2.

Backport-of: #1844

Checklist
  • npm install && npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

Python 3 is allowed as a compatible Python, but its looked for after
Python 2.

Backport-of: #1844
@joaocgreis
Copy link
Member

#1909 includes EXPERIMENTAL_NODE_GYP_PYTHON3. The second PR to land should remove it from there.

rvagg pushed a commit that referenced this pull request Oct 4, 2019
Python 3 is allowed as a compatible Python, but its looked for after
Python 2.

rvagg: removed additional EXPERIMENTAL_NODE_GYP_PYTHON3 in
.travis.yml from a previous commit while landing.

Backport-of: #1844
PR-URL: #1910
Reviewed-By: Christian Clauss <[email protected]>
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: João Reis <[email protected]>
@rvagg
Copy link
Member

rvagg commented Oct 4, 2019

sweet, landed in v5.x with additional EXPERIMENTAL_NODE_GYP_PYTHON3 removed

@rvagg rvagg closed this Oct 4, 2019
@rvagg rvagg mentioned this pull request Oct 4, 2019
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg pushed a commit that referenced this pull request Oct 4, 2019
Python 3 is allowed as a compatible Python, but its looked for after
Python 2.

rvagg: removed additional EXPERIMENTAL_NODE_GYP_PYTHON3 in
.travis.yml from a previous commit while landing.

Backport-of: #1844
PR-URL: #1910
Reviewed-By: Christian Clauss <[email protected]>
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: João Reis <[email protected]>
@sam-github sam-github deleted the allow-py3-after-py2 branch October 25, 2019 14:43
@cclauss cclauss added the Python label Nov 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants